headerbar: Respect the show-close-button property

window: Update decorations when updating state

This fixes the maximize button not updating...

testtitlebar: Fix initial values

Make sure that the controls show the initial values fo the
headerbar properties, not the other way around.

testtitlebar: more tests

Turn this into a GtkApplication with an app menu.
Allow to override the gtk-shell-shows-app-menu setting
and the decoration-button-layout style property.

headerbar: Support all kinds of CSD decorations

Move the gtkwindow.c CSD code into GtkHeaderBar, and make it triggerable
by the show-close-button property, and remove shows-fallback-app-menu.

https://bugzilla.gnome.org/show_bug.cgi?id=720233

GtkHeaderBar: treat "" as 'no subtitle'

This is one of the few cases where it makes some sense to blur
the line between and empty string and NULL: without this, it is
hard to reset the subtitle e.g. from a builder file. And we
have the has-subtitle property now to enforce subtitle size
allocation independently.

GtkFileChooser: Don't fill location_entry if we just populated the file list

Only fill the location entry with the file name of the tree view's
selected file when the selection was done by the user.

When the file chooser's action is GTK_FILE_CHOOSER_ACTION_OPEN, it
selects the first file in the tree view once loading has finished. For
this case we don't want it to insert the file name in the location
entry, as it hinders efficient navigation using the location entry. To
achieve this, use a priv flag to keep track of whether the
selection-changed signal was caused by the file chooser itself.

https://bugzilla.gnome.org/show_bug.cgi?id=386569

Make testtitlebar more testy

Now that the nice titlebar example is in gtk3-demo, we can
use testtitlebar as an actual testbed for headerbar stuff.

This immediately reveals size allocation issues when titlebar
widgets change size.

gtk-demo: Use title case consistently

Add a header bar example to gtk-demo

This is basically standalone the testtitlebar example, turned
into a demo.

Updated Spanish translation

testtitlebar: Fix -Wunused-variable warning

GtkWidget *title is no longer used.

https://bugzilla.gnome.org/show_bug.cgi?id=720296

GtkFileChooserDefault: Remove unused struct field

The field changing_folder is no longer used anywhere.

https://bugzilla.gnome.org/show_bug.cgi?id=720293

headerbar: Remove unused private API

Don't remove gtkheaderbarprivate.h, even though it's empty,
since we'll add more to it soon.

https://bugzilla.gnome.org/show_bug.cgi?id=720233

wayland: Fix order of xdg-shell requests

Updated Hebrew translation

Updated Galician translations

Gettext comment for "default:LTR" must be before the string

https://bugzilla.gnome.org/show_bug.cgi?id=720096

Simplify a check

Whenever titlebar is set, title_box is pointing to the
same widget, so we can just always use title_box here.
Pointed out by Jasper St. Pierre.

Trivial typo fix

Use ::has-subtitle in testtitlebar

This also gets simplified.

Use ::has-subtitle when constructing CSD titlebars

GtkWindow was constructing a custom title label to
avoid the size reservation for a subtitle. This is
now easier with the ::has-subtitle property.

GtkHeaderBar: Add a ::has-subtitle property

It is a fairly common case to just want a title, and not
reserve extra space for a subtitle. This is much easier
to get right by setting a boolean property than by
constructing a custom title widget.

Fix a segfault in the event axes demo

Seems that pointer_info can be NULL, after all.

Update headerbar title from window title

This removes a possible source of confusion in our API.
The title is now synchronized both ways between the window
and the headerbar.

https://bugzilla.gnome.org/show_bug.cgi?id=720067

Sync the window title initially

When setting a custom titlebar that happens to be a GtkHeaderBar,
we connect to notify::title to pick up title changes on the headerbar,
but we forgot to sync the title initially. Fix that.

Disconnect signal handlers

We connect to notify::title on app-provided titlebars, but
we forgot to disconnect the signal handler when unsetting
the titlebar.

Add a function to determine the text direction

This function can be used in the rare cases where the locale
has to be changed after gtk_init(). Based on a patch
by Jehan <jehan@girinstud.io>.

https://bugzilla.gnome.org/show_bug.cgi?id=720096

demo: only set transient if demo returns a toplevel

demo: put run button on the left

It is a bit annoying to have to go over to the right side to click it.

wayland: Update to latest xdg-shell.xml

wayland: Make sure to call use_unstable_version

button: Allow :hover on insensitive buttons

You can still hover a mouse on insensitive elements; it's up to the
theme to disable that.
This is in line with the HTML/CSS interpretation of :hover.

Insensitive elements still cannot be clicked.

https://bugzilla.gnome.org/show_bug.cgi?id=719486

gdk: Add some return_if_fail() warnings

so we have a better chance of catching the correct cause of bugs like

https://bugzilla.gnome.org/show_bug.cgi?id=719977

filechooser: Don't crash if invalid thumbnails exist

Failing to load a thumbnail returns a NULL pixbuf. Since the hidpi
patches this wasn't checked when creating the surface. Result: assertion
failure.

https://bugzilla.gnome.org/show_bug.cgi?id=719977

gtkradio*: add (allow-none) annotations were appropriate

https://bugzilla.gnome.org/show_bug.cgi?id=679563

demo: Add demo for advanced event information management

This demo condenses the essentials of advanced management of
input events. Depending on the information available in input events,
this demo will try to represent as much information as possible for
those.

https://bugzilla.gnome.org/show_bug.cgi?id=719987

docs: Add a missing Since: tag

Don't distribute generated sources

This was causing problems when building 3.10.6 against an older
wayland.

GtkMenuItem: only set 'separator' class on the separator line

To make the separator line distinguishable from the widget's normal
border from css.

https://bugzilla.gnome.org/show_bug.cgi?id=719861

Fix make dist

x11: Handle XINotifyPassiveGrab/Ungrab in focus events

The focus handling code is shared between core and XI2 implementations,
so just handle the extra XI2 types for passive grabs. Those must be dealt
with in the same way than active grabs. Focus events with this crossing
mode could happen currently through the XIGrabFocusIn passive grab.

https://bugzilla.gnome.org/show_bug.cgi?id=719762

x11: Handle all XI2 crossing mode values in switch

This fixes potential assertions if a GTK+ app gets to receive
a XINotifyPassiveGrab/Ungrab pointer crossing event, currently
triggerable by XIGrabEnter passive grabs.

http://bugzilla.gnome.org/show_bug.cgi?id=719762

recentchooserdefault: Clean up load_id when we have nothing left to do

Returning FALSE from a GSourceFunc will implicitly remove it, so we need
to do this in order to make the new g_source_remove happy.

Updated POTFILES.skip

window: enable client side decorations deactivation with wayland backend

Revert "gdkwindow-x11: Don't set WM_TAKE_FOCUS if accept_focus is FALSE"

This reverts commit 809aab2c8ec88c7b10254a948ba8754a766b1f7c.

Bug 712536 - Themes with gap_(start|end)_file but no actual file crash

Code factorization in commit 34fd123 reintroduced bug fixed in 0d396ab
with non-equivalent factorized tests.

treeview: Don't draw a background for the bin window

The bin window's background would have to be drawn in the bin window's
size and inside the pixel cache draw function to not cause transparency
issues.

But because it's unnecessary as the view window draws the same
background, we just skip it.

https://bugzilla.gnome.org/show_bug.cgi?id=709027

Tajik translation updated

gtk-demo: fix colorchooser example

A nested dialog on top of a modal dialog must itself be modal,
otherwise it doesn't get any input.

GDK-Win32: Ensure sources use UNIX line endings

This makes things more consistent with the other sources and makes patches
easier to apply cleanly.

MSVC Projects: Use UNIX Line endings

...for all files except the README.txt and the .sln files, which have to
have DOS/Windows line endings.  This makes application of patches, when
applicable, easier.

file-chooser: add a sort-directories-first gsetting

Add an option to sort folders before files in order to be
more constistent with the file manager.

demo: Fix the size of the run button

demo: make demo windows transient to the main window

This prevents them getting lost behind.

demo: Add a GtkHeaderBar

demo: change application name of inner demo

demo: expand notebook tabs

demo: Port to GtkApplication

And use GtkBuilder to construct interface.

gtk object tests: run under local environment

Make sure that the tests don't access the host's session bus or
installed gsettings schemas.

Also disable tests for some classes that leak a connection to the
session bus.

https://bugzilla.gnome.org/show_bug.cgi?id=711715

toolbar: draw a full box for separators

This allows themes do whatever they want as separators, with
paddings, borders and backgrounds.

If "wide-separators" property is true, then, instead of just draw
a frame, also render its background, and take into account the
padding property for its limits.

https://bugzilla.gnome.org/show_bug.cgi?id=719713

button: Call the new function

This simplifies a lot of code and doesn't require hacks setting
in_button anymore.

button: Split out function

This is in preparation of the following cleanup commits.

button: Don't emit a signal on broken grabs

We don't emit one on grab notify, so we don't want one for broken grabs
either.

gdkwindow: Remove an unused assignment

The show variable is assigned to in both branches of an if block a
little further down.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gdkwindow: Remove an unused assignment

scale is only used if HAVE_CAIRO_SURFACE_SET_DEVICE_SCALE is defined.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkprintunixdialog: Remove a redundant (GFile != NULL) check

g_file_new_for_uri() is guaranteed to return a non-NULL value, so this
check was redundant, and was confusing the static analyser into
returning a false positive, where it thought the file could be NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Eliminate a dead assignment

This is technically a dead assignment, but is nice to retain for
clarity. Moving it to the variable definition shuts scan-build up.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkthemingengine: Eliminate a dead assignment

This is technically a dead assignment, but is nice to retain for
clarity. Moving it to the variable definition shuts scan-build up.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkthemingengine: Fix a definite use of an uninitialised variable

At this point, segments[1] is always uninitialised, and is used to
initialise itself. Looking at the code in the branch above, this appears
to have been a typo from segments[0], as segments[1] seems to typically
be 2 * segments[0].

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkpapersize: Reformat #if preprocessor commands

This eliminates some false positive warnings from scan-build, which was
not interpreting the #ifs and hence warning about unbalanced #endifs.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkmenu: Fix potential use of uninitialised variables

The child_height out variable is only valid if compute_child_offset()
returns TRUE.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeviewaccessible: Fix potential uses of uninitialised variables

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

GtkRevealer: Fix size allocation with disabled animation

When animation is disabled, we use 'none' as the effective
transition type. So far, this transition type failed to change
the size request, causing the revealer to always take up the
space of the child, even when the child is not shown.

https://bugzilla.gnome.org/show_bug.cgi?id=719568

Fix the AtkImage implementation for GtkIconView items

We were returning uninitialized values for image size and
position.

Found in a pile of patches in
https://bugzilla.gnome.org/review?bug=712760

Code cleanup

Don't compare booleans to FALSE.

annotations: Set GtkTreeModelFilterModifyFunc value argument to out

Use (out caller-allocates) annotation for the "value" argument to
GtkTreeModelFilterModifyFunc. This is needed because GI based language
bindings coerce GValue input args into native types and there is no
opportunity to set the value within the GValue itself.

https://bugzilla.gnome.org/show_bug.cgi?id=719460

Add class toolbar before the style context is created for the widget.

The call to gtk_button_set_relief() in gtk_toolbar_init() indirectly
used the style context of the half-created widget, before we had a
chance to add the "toolbar" style class to it.

Reorder gtk_toolbar_init() to ensure that the proper style class is
set first.

https://bugzilla.gnome.org/show_bug.cgi?id=719595

infobar: Fix visibility when enable-animations=false

When enable-animations is false, the revealer's child-revealed property is
notified immediately, so make sure to connect to it before toggling the
revealer.

https://bugzilla.gnome.org/show_bug.cgi?id=719510

Correct the deprecation annotation for gtk_widget_modify_bg

It was pointing to a nonexisting function.

https://bugzilla.gnome.org/show_bug.cgi?id=719469

Don't use depeprecated api

GtkPrintUnixDialog and testtitlebar were still using
gtk_widget_set_margin_left/right. Switch them over to
gtk_widget_set_margin_start/end.

message dialog: Make labels wrap at a reasonable length

Several labels in GtkMessageDialog were set to wrap, but
did not set max-width-chars. Fix that.

about dialog: Make labels wrap at reasonable length

Several labels in GtkAboutDialog are set to wrap, but
don't set a value for max-width-chars. Fix that.

places sidebar: Use a symbolic desktop icon

This was pointed out in

https://bugzilla.gnome.org/show_bug.cgi?id=719505

build/win32/vs*/README.txt: Re-word a bit

Use PATH rather than path to make things clearer

GtkAboutDialog: fixed calling gtk_about_dialog_set_logo_icon_name()
with a NULL icon_name. Now it will actually use the default
window icon set as mention in the docs

GtkIconCache: find_image_offset() return 0 if icon_name is NULL.
This avoid a crash calling gtk_about_dialog_set_logo_icon_name() with a NULL icon_name.

Fix CSD check for Win32

Win32 does not have alpha channel currently ; fix the check
for this, so trying to enable CSDs on this platform will
not "succeed" and crash the app anymore.

Partially fixes gtk3-widget-factory.

Visual C++ Projects: Add rules to generate GTK DBus sources

Since commit 7c2a5072 the gtkdbusgenerated.[c|h] are not included in the
dist tarball and thus have to be generated, which broke the Visual C++
builds.

This patch adds property sheets and custom build rules for the Visual C++
projects so that gtkdbusgenerated.[c|h] will be generated upon building the
GTK+ DLL sources.

This also tells people building GTK+ from the projects that they need to
have Python 2/3 installed and the Python interpretor needs to be in their
PATH before building GTK+ from the projects.

Update GDK-Broadway MSVC Projects

-Improve optimization a bit for broadwayd, by enabling link time code
 generation
-Add PlatformToolset tag for the Visual C++ 2010 projects, to ease
 transition to Visual C++ 2012/2013

Tamil Translations Updated

Don't distribute gdbus-generated sources

These are supposed to be regenerated at build time,
to avoid dependencies on the dist-systems glib version.

defaultvalue test: ignore show-desktop

Ignore the "show-desktop" property on GtkPlacesSidebar for the
defaultvalue test.

Currently, "make check" is passing because it runs the test under a xvfb
with no XSETTINGS provider, so we see the Gtk default value.  No matter
what we set the default value to in Gtk, however, there will be some
desktop environment in which someone running the installed test outside
of an xvfb will get the wrong result.  Best to ignore it.

https://bugzilla.gnome.org/show_bug.cgi?id=712302

Change the default for "show-desktop" back to TRUE

Change the GtkSettings default for "shell-shows-desktop" back to TRUE
and also change the default value of the "show-desktop" property on
GtkPlacesSidebar so that the defaultvalue test passes.

https://bugzilla.gnome.org/show_bug.cgi?id=712302

bindings: Fix stray semicolons in CSS example for bindings

Semicolon shouldn't be at the end of binding set

x11: Don't crash if a window is on an unmanaged screen

_gdk_x11_display_screen_for_xrootwin() can return NULL. If so, return
NULL from gdk_x11_window_foreign_new_for_display().

https://bugzilla.gnome.org/show_bug.cgi?id=709716

x11: keep track of the screen pixel size by calculating the bounding box of monitors

This is so we always have the latest information given by XRandR (or other), and not
rely on Core protocol information that might not have been updated yet.

This is specially visible when a monitor is connected (less frequent) or disconnected
(much more frequent), callbacks on GdkScreen::monitors-changed that call
gdk_screen_get_width/height() could get the screen size previous to the monitor
rearrangement.

So in order to fix this, keep track of the latest monitors information, and calculate
the bounding box in order to know the screen size.

https://bugzilla.gnome.org/show_bug.cgi?id=715029

printing: Fix memory leak when listing Avahi printers

Return values of g_variant_get_child_value() were not unreffed
correctly together with one value returned by g_variant_get().
Use g_variant_get_data() instead of copying each byte separately.

https://bugzilla.gnome.org/show_bug.cgi?id=712799

Broadway - Fix UNIX/Win32 detection

Finally fix this check so unnecessary headers aren't needed.